Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(2P): rewrite MIP constraint to respect relative range limits #1090

Merged
merged 1 commit into from
Jul 12, 2024

Conversation

MartinBelthle
Copy link
Collaborator

@MartinBelthle MartinBelthle commented Jul 11, 2024

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Does this PR already have an issue describing the problem?
No

What kind of change does this PR introduce?
Bug fix

What is the current behavior?
Due to the recently merged PR #1070, the relative ranges for PST in 2P Global Optimization (when modeled as APPROXIMATED_INTEGERS) were no longer respected.

This made the test 19.11.9.bis fail when ran with XPRESS.

What is the new behavior (if this is a feature change)?
I've changed the MIP equation and now they are.

Does this PR introduce a breaking change or deprecate an API?

  • Yes
  • No

@MartinBelthle MartinBelthle added the bug Something isn't working label Jul 11, 2024
@MartinBelthle MartinBelthle self-assigned this Jul 11, 2024
@MartinBelthle MartinBelthle force-pushed the fix/add-back-relative-range-limitations-for-2P branch 2 times, most recently from 884d600 to 769732d Compare July 11, 2024 12:37
@MartinBelthle MartinBelthle changed the title fix(2P): rewrite MIP equation to respect relative range limits fix(2P): rewrite MIP constraint to respect relative range limits Jul 11, 2024
@pet-mit pet-mit added next-release This issue or pull request should be resolved before next release PR: waiting-for-review This PR is waiting to be reviewed labels Jul 11, 2024
@MartinBelthle MartinBelthle force-pushed the fix/add-back-relative-range-limitations-for-2P branch from 5104576 to 018b978 Compare July 12, 2024 09:00
@pet-mit pet-mit merged commit 7ce2fc3 into main Jul 12, 2024
10 checks passed
@pet-mit pet-mit deleted the fix/add-back-relative-range-limitations-for-2P branch July 12, 2024 09:12
@pet-mit pet-mit mentioned this pull request Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working next-release This issue or pull request should be resolved before next release PR: waiting-for-review This PR is waiting to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants